Skip to content

Conversation

@evilkermit
Copy link
Member

@evilkermit evilkermit commented Sep 17, 2025

Summary of Changes

Adds the requested examples from https://code.ornl.gov/groups/ndip/-/epics/252.

Checklist

  • The PR has a clear and concise title
  • Code is self-documented and follows style guidelines.
  • Automated tests are written and pass successfully.
  • Regression tests (e.g. manually triggered system tests, manual GUI/tool tests, ...) are performed to make sure the PR does not break anything (when applicable)
  • Readme file is present and up-to-date.

Documentation Updates

Additional Notes

Closes #7
Closes #8

@evilkermit evilkermit self-assigned this Sep 17, 2025
@evilkermit evilkermit linked an issue Sep 17, 2025 that may be closed by this pull request
@evilkermit evilkermit changed the title Add conditional render/disable examples Add requested examples Sep 17, 2025
@evilkermit evilkermit requested a review from jarrah42 September 22, 2025 17:43
# circumstances.
NeutronDataSelector(
v_model="data.selected_files",
base_paths=["/HFIR", "/SNS"],

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would specify something that you know will exist, such as $HOME and /tmp.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wait... is this an example of DataSelector or NeutronDataSelector? You should have one of each.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll create one just for the DataSelector and set some paths as you mentioned.

README.md Outdated
4. [Conditionally disabling elements](examples/conditional_disabling)
5. [Changing Pydantic rules based on user input](examples/dynamic_pydantic_rules)
6. [Complex Pydantic rules](examples/complex_pydantic_rules)
7. [Using datafiles from the Analysis Cluster](examples/data_selector)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we move this to a separate ORNL Only section?, ditto ONCat.

README.md Outdated
6. [Complex Pydantic rules](examples/complex_pydantic_rules)
7. [Using datafiles from the Analysis Cluster](examples/data_selector)
8. [Selecting datafiles from ONCat](examples/oncat)
9. [Running an NDIP tool](examples/run_ndip_tool)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change NDIP to Galaxy

with vuetify.VCard(classes="mx-auto my-4 pa-1", max_width=600):
with HBoxLayout(classes="mb-2", gap="0.5em", valign="center"):
InputField(v_model="data.input_path")
with vuetify.VBtn(

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why VBtn?

@evilkermit
Copy link
Member Author

@jarrah42 I've updated the PR. Created an ORNL-specific subdirectory for examples and added an example for the plain DataSelector. Let me know if you want to see any other changes!

Copy link

@jarrah42 jarrah42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add programmatic disable/enable Add programmatic visibility example

3 participants